Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: bump highcharts/dashboards to v2, resolve #22 and #6 #44

Merged
merged 5 commits into from
Jun 2, 2024

Conversation

volatile-static
Copy link
Contributor

@volatile-static volatile-static commented Jun 2, 2024

resolve: #22
resolve: #6

@volatile-static
Copy link
Contributor Author

不知道是不是我用npm更新依赖有问题,这个lock报错我没太搞懂

@northword
Copy link
Member

应该是你用的 npm,锁文件是package-lock.json,没有更新 pnpm 的锁文件 pnpm-lock.yaml,而仓库 ci 用的pnpm

@northword northword changed the title resolve #22 and #6 fix: bump highcharts/dashboards to v2, resolve #22 and #6 Jun 2, 2024
@northword
Copy link
Member

这个合并后可以 revert zotero-chinese/zotero-plugins@c5c611b 嘛?

@volatile-static
Copy link
Contributor Author

我这边是可以的,建议你也试一下

northword added a commit to zotero-chinese/zotero-plugins that referenced this pull request Jun 2, 2024
northword added a commit to zotero-chinese/zotero-plugins that referenced this pull request Jun 2, 2024
Copy link

github-actions bot commented Jun 2, 2024

✅ 成功部署预览!

类型 链接
预览地址 https://deploy-preview-44--zotero-zh.netlify.app
最新提交 1e6ffa6
部署日志 部署预览 #69

@northword
Copy link
Member

looks good!

@northword northword merged commit 6240791 into zotero-chinese:main Jun 2, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

插件图表:图丢失
2 participants